Skip to content

GitHub App: faster migration page #12186

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 27, 2025
Merged

GitHub App: faster migration page #12186

merged 3 commits into from
May 27, 2025

Conversation

stsewd
Copy link
Member

@stsewd stsewd commented May 19, 2025

For some things, we are returning lists of objects, so no lazy iteration is possible for those. So I'm loading those only when the proper step needs them.

@stsewd stsewd requested a review from a team as a code owner May 19, 2025 17:51
@stsewd stsewd requested a review from humitos May 19, 2025 17:51
stsewd added a commit to readthedocs/ext-theme that referenced this pull request May 19, 2025
@stsewd stsewd merged commit 70b4e5a into main May 27, 2025
4 of 5 checks passed
@stsewd stsewd deleted the faster-migration-page branch May 27, 2025 17:01
stsewd added a commit to readthedocs/ext-theme that referenced this pull request May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants